Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: silent flag in .air.toml, suppressing all air prints #641

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

jesses-code-adventures
Copy link
Contributor

@jesses-code-adventures jesses-code-adventures commented Sep 7, 2024

useful if the app being watched has structured outputs that are being piped into subsequent program/s.

@silverwind
Copy link
Contributor

silverwind commented Sep 12, 2024

Will errors still show with this flag enabled? If so, it should be fine imho. Errors should never be suppressed imho.

@jesses-code-adventures
Copy link
Contributor Author

this library generally uses log.Fatalf() for errors, whereas these changes only affect info and debug logs in runner/util.go and runner/logger.go which produce the lion's share of the logs. shouldn't be anything to worry about as far as errors being suppressed.

@cosmtrek
Copy link
Collaborator

@jesses-code-adventures Hi, thanks for adding the silent flag, could you rebase the master and rerun the checks?

useful if the app being watched has structured outputs that are being
piped into subsequent program/s.
@jesses-code-adventures
Copy link
Contributor Author

jesses-code-adventures commented Sep 25, 2024

@cosmtrek i have rebased the master but i think I need maintainer approval to run checks

Copy link

codecov bot commented Sep 25, 2024

Codecov Report

Attention: Patch coverage is 8.69565% with 21 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
runner/util.go 0.00% 6 Missing and 6 partials ⚠️
main.go 0.00% 7 Missing ⚠️
runner/logger.go 0.00% 1 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
runner/config.go 79.47% <100.00%> (+0.93%) ⬆️
runner/logger.go 65.30% <0.00%> (-2.78%) ⬇️
main.go 0.00% <0.00%> (ø)
runner/util.go 72.53% <0.00%> (-3.94%) ⬇️

@cosmtrek cosmtrek merged commit 0dfcd29 into air-verse:master Sep 25, 2024
7 of 8 checks passed
@jesses-code-adventures jesses-code-adventures deleted the silent_config_flag branch October 5, 2024 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants